-
-
Notifications
You must be signed in to change notification settings - Fork 493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Allow to configure the Elasticsearch connection in environment variables #8585
base: main
Are you sure you want to change the base?
Conversation
|
@josegar74 - I've noticed some inconsistencies in the configuration (nothing to do with this PR). By this I mean;
So, you must:
I'm not quite sure what will happen if this is run in a WAR - I don't think I'm not quite sure how to fix this so its consistent and doesn't affect current installations. |
@josegar74 - I created this PR (to this PR) that fixes those inconsistencies. However, you can probably just merge this with just a doc change: Note that the individual components AND |
@davidblasby about es.url, the documentation had an example, maybe it should be indicated more clearly?
|
Co-authored-by: david.blasby <[email protected]>
This pull request allows to configure the Elasticsearch connection using environment variables.
Checklist
main
branch, backports managed with labelREADME.md
filespom.xml
dependency management. Update build documentation with intended library use and library tutorials or documentation